文字コードの指定ダイアログのCPチェックの仕様を修正する #1059
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
✅ Build sakura 1.0.2278 completed (commit f39b53ddc4 by @berryzplus) |
|
レビューありがとうございます。 |
HoppingTappy
pushed a commit
to HoppingTappy/sakura
that referenced
this pull request
Jun 16, 2020
…charset 文字コードの指定ダイアログのCPチェックの仕様を修正する
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
文字コードの指定ダイアログのCPチェックの仕様を修正することにより、
issue #1037 の不具合(CPチェックが効かないことがある)を解消します。
カテゴリ
PR の背景
issue #1037 の対策PRです。
対応する不具合内容
不具合の原因
IDC_CHECK_CPを利用する他4つのダイアログと実装が異なっている#1037 (comment) でコメントしている通り、不具合を解消するだけならずっと少ない修正で事足ります。
この PR では、不具合原因のうち 2. の「他4つのダイアログと実装が違う」に着目し、実装を合わせてみました。
CPチェックの状態を格納するメンバ変数を削除し、
IDC_CHECK_CPのイベント実装をシンプルにしました。PR のメリット
issue #1037 の不具合が解消します。
PR のデメリット (トレードオフとかあれば)
とくにありません。
なお、Windows API CheckDlgButton の使い方が誤っている件についてはスルーします。
※CheckDlgButton関数の第3引数はBOOLではなくintです。TRUEで動作している理由は、BST_CHECKEDの値が1でTRUEの定義とたまたま一致しているためです。
PR の影響範囲
関連チケット
close #1037 文字コードの指定ダイアログでCPのチェックが効かない事が有る
参考資料